Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix: show the default validation message when there is a required error #386

Merged
merged 2 commits into from
Jan 18, 2017

Conversation

wuct
Copy link
Contributor

@wuct wuct commented Sep 1, 2016

No description provided.

@wuct
Copy link
Contributor Author

wuct commented Sep 1, 2016

Might related to #276.
However, showError() is still buggy.

@wuct
Copy link
Contributor Author

wuct commented Sep 1, 2016

OK, I manage to fix showError() by this commit .

@valoricDe
Copy link

@christianalfoni Works for me.

@aesopwolf aesopwolf merged commit ac301f8 into christianalfoni:master Jan 18, 2017
@aesopwolf
Copy link
Collaborator

Thanks @wuct, just published v0.19.1 with this in it https://www.npmjs.com/package/formsy-react

@wuct
Copy link
Contributor Author

wuct commented Jan 19, 2017

Thanks for merging

aesopwolf added a commit that referenced this pull request Jan 24, 2017
sdemjanenko added a commit to meraki/mki-formsy-react that referenced this pull request May 26, 2017
* christ/master:
  v0.19.2
  Revert "fix: show the default validation message when there is a required error (christianalfoni#386)"
  v0.19.1
  fix: show the default validation message when there is a required error (christianalfoni#386)
  Remove stray sentence. (christianalfoni#415)
  Update API.md
  Fix crash when comparing array with null (christianalfoni#404)
  docs: Removing unnecessary single quote (christianalfoni#410)
  0.19.0
  feat: expose inner element when using HOC (christianalfoni#399)
  Update README.md
  Update LICENSE
  Nit-picky typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants